geekha/refactor footer #12
Conversation
add sizes props to reuse in hero and footer
change design footer and implement in home page
src/app/page.tsx
Outdated
| @@ -94,7 +95,7 @@ export default function Home() { | |||
| ))} | |||
| </div> | |||
| </main> | |||
| <footer className="relative z-10 p-4 font-koulen text-white"> | |||
| {/* <footer className="relative z-10 p-4 font-koulen text-white"> | |||
There was a problem hiding this comment.
Creo que sería mejor borrar el código que no usaremos y dejar solo el Footer, qué opinas?
| height?: string; | ||
| } | ||
|
|
||
| export const Title = ({ width = "100%", height = "100%" }: propsType) => ( | ||
| <svg |
There was a problem hiding this comment.
👍 Genial, lo agregué en los nuevos iconos.
|
En general se ve bien Pero voy a tener que probarlo localmente para poder ver la UI. Mi recomendación personal (que debo agregar como template al repo) es agregar un par de screenshots cuando hayan cambios visuales para así tener una idea. Respecto a tus comentarios:
|
|
He agregado unas mejoras para finalizar el footer dejo las captures de como quedó: Versión desktop: Versión mobile
|
Hay que refactorizarlo cuando se integre el Sanity, pero como está creo que cumple para el merge. |
|
Se elimina antiguo footer en |
|
Esto se continuó en el PR: #36 Con @geekhadev como contribuidor interno. |
Se envía primera fase de la PR para revisión.
Hay dos temas que no supe resolver en primera instancia:
Si hay ideas les agradezco no soy experto en FrontEnd.